Don't free a list that has not been initialized
authorMatthias Clasen <mclasen@redhat.com>
Wed, 10 Apr 2013 01:18:05 +0000 (21:18 -0400)
committerMatthias Clasen <mclasen@redhat.com>
Wed, 10 Apr 2013 01:18:05 +0000 (21:18 -0400)
modules/printbackends/cups/gtkprintbackendcups.c

index 80d868c83d59f5cc5fd491dd24d153e92769ddfd..34c6eac3e53f622033dda5ba5107b0490a01a7af 100644 (file)
@@ -2765,10 +2765,6 @@ avahi_service_browser_signal_handler (GDBusConnection *connection,
     }
   else if (g_strcmp0 (signal_name, "ItemRemove") == 0)
     {
-      GtkPrinterCups *printer;
-      GList          *list;
-      GList          *iter;
-
       g_variant_get (parameters, "(ii&s&s&su)",
                      &interface,
                      &protocol,
@@ -2780,6 +2776,10 @@ avahi_service_browser_signal_handler (GDBusConnection *connection,
       if (g_strcmp0 (type, "_ipp._tcp") == 0 ||
           g_strcmp0 (type, "_ipps._tcp") == 0)
         {
+          GtkPrinterCups *printer;
+          GList          *list;
+          GList          *iter;
+
           list = gtk_print_backend_get_printer_list (GTK_PRINT_BACKEND (backend));
           for (iter = list; iter; iter = iter->next)
             {
@@ -2799,9 +2799,9 @@ avahi_service_browser_signal_handler (GDBusConnection *connection,
                   break;
                 }
             }
-        }
 
-      g_list_free (list);
+          g_list_free (list);
+        }
     }
 }